Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Notarise the macOS app #11119

Merged
merged 8 commits into from
Oct 11, 2019
Merged

Notarise the macOS app #11119

merged 8 commits into from
Oct 11, 2019

Conversation

dbkr
Copy link
Member

@dbkr dbkr commented Oct 11, 2019

Notarise the mac app in the electron build process

@dbkr dbkr requested a review from a team October 11, 2019 10:03
Copy link
Contributor

@bwindels bwindels left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm!

// We get the password from keychain. The keychain stores
// user IDs too, but apparently altool can't get the user ID
// from the keychain, so we need to get it from the environment.
const userId = process.env.NOTARIZE_APPLE_ID;
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Who will set this env var? Can we manually extract it from keychain with keychain find-generic-password ... or similar?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I was just going to put it in the ~/.profile on the build machine. I'd hoped we'd be be able to just get it out of the keychain but I can't work out how to get the entry in a place in the keychain where both altool and security can find it...

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Aha, I see... Well, probably it's fine. 😄

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What's the current method for getting this set up? I've tried NOTARIZE_CREDS=foo yarn build:electron:macos to no avail...

Copy link
Collaborator

@jryans jryans Jan 29, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's discuss this in #riot-dev.

@turt2live
Copy link
Member

(@dbkr can haz a title that will look okay in the changelog please?)

@dbkr dbkr changed the title Dbkr/notarise Notarise the macOS app Oct 11, 2019
@dbkr dbkr merged commit 21ea6c6 into develop Oct 11, 2019
@t3chguy t3chguy deleted the dbkr/notarise branch May 12, 2022 09:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants